Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use buffer hole erosion heuristic for rings #1117

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

dr-jts
Copy link
Contributor

@dr-jts dr-jts commented Feb 5, 2025

Applies the hole erosion heuristic from #706 to the inside offset curve of buffers of linear rings.

This fixes the issue reported in libgeos/geos#1223.

LINESTRING(25 44, 31 44, 32 38, 29 37, 25 37, 25 38, 24 40, 24 44, 25 44)
image

@dr-jts dr-jts added the type-bug label Feb 5, 2025
@dr-jts dr-jts changed the title Fix ring buffer hole erosion Add ring buffer hole erosion heuristic Feb 5, 2025
@dr-jts dr-jts changed the title Add ring buffer hole erosion heuristic Use buffer hole erosion heuristic for rings Feb 5, 2025
@dr-jts dr-jts merged commit 040802a into locationtech:master Feb 5, 2025
2 checks passed
@dr-jts dr-jts deleted the fix-ring-hole-erosion branch February 5, 2025 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant